-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[REVIEW] Update Dockerfile for setup #27
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My only concern here is that we might want the latest dmlc xgb and not the rapids version.
That's a good point, I can undo the xgboost install step. Would this require installing from source, or the latest stable version (using |
I think it should be source. |
@RAMitchell I've updated the dockerfile but the xgboost install isn't working with the following error:
I can run the same command within the container, but the Dockerfile for some reason fails. Do you know what could be the reason for this? |
I don't I'm afraid. Maybe @hcho3 has an idea? |
|
@hcho3 is it However, this worked when I ran it inside
|
Can you run |
Thanks! That helped |
The previous version of the file used older version of RAPIDS and didn't build properly for me.
Changing it to use RAPIDS image instead of nvidia/cuda image as most of these packages come pre-installed including xgboost and light-gbm.